Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rendered Jinja params for system props in control implementation statements #3

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

gregelin
Copy link
Collaborator

@gregelin gregelin commented Dec 10, 2023

This PR adds support for Jinja parameter substitution in component control implementations. The support is currently only for system object parameters.

Purpose is to be able to include the System Title (and other system-specific information) within a control implementation statement that rendered in the System Security Plan.

Change accomplished by selectively adding a statement.body_rendered property. The body_rendered property is added in two places:

  1. when rendering the read-only system control implementation statements that are displayed for system controls,
  2. when rendering the Statement's control_implementation_as_dict's combined statement that is used to display the control implementation statements in the System Security Plan output templates.

This approach nicely allows components to contain the Jinja parameter double brace template parameters (e.g., {{ system.root_element.title }}).

Todo:

  • check if rendering is working system component control implementation statements

Screenshots

in the System control implementations statements

image

in the System control implementation statement editing, the Jinja double brace parameter still appears

image

In the rendered SSP

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant